You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Removal of Activity Logging
Previous versions of our software kept a detailed log of the creation, updating, and deletion actions pertaining to the Report Template model. These logs have now been removed to streamline our processes and maximize efficiency.
Introduction of a New Migration
A new migration process has been introduced to clean any previous logs related to the Report Template model's activity. This process will retrospectively clean up our system and improve storage efficiency.
Deletion of Redundant Tests
As we've removed the logging functionalities associated with the Report Template model, any tests that were specifically designed to examine these functionalities are no longer required. As such, the 'ReportTemplateActionLogTransformerTest' and 'ReportTemplateActivityLoggingTest' have been deleted, reducing the clutter and complexity of our testing procedures.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
marcusmoore
deleted the
chore/custom-report-template-activity-log-removal
branch
THIS PR DELETES DATA FROM THE ACTIVITY LOG
@snipe @uberbrady the query in the migration seems simple enough but please try to think of any ways it could go wrong and delete extra data.
This PR stops the reporting of
ReportTemplatecreates, updates, and deletes to the activity log and purges existing activity log records.For reference: reporting was originally included in #15714
Fixes #16417